Skip to content

Python docs + a small C++ selection, done by hand#20

Open
simonevadi wants to merge 10 commits into
mainfrom
split/docs
Open

Python docs + a small C++ selection, done by hand#20
simonevadi wants to merge 10 commits into
mainfrom
split/docs

Conversation

@simonevadi

Copy link
Copy Markdown

This MR sets up the initial documentation infrastructure using Read the Docs.

It includes:

  • Read the Docs configuration and Sphinx setup.
  • An initial Installation guide.
  • A first Tutorial.
  • A manually written API reference.

At the moment, the API documentation is maintained by hand. The next step will be to automate the generation of the Python API documentation (e.g., via autodoc or a similar approach).

Additionally, this MR adds a helper script to build and preview the documentation locally:

bash tools/build_docs.sh --html --open

The script creates a dedicated virtual environment for the documentation, installs the required dependencies, builds the HTML documentation, and serves the output from docs/_build/html/.

simonevadi and others added 9 commits June 29, 2026 12:27
- examples/tutorial_tracing.py: runnable example using tests/res/test.rml,
  verified against installed rayx 0.4.3
- event_type column is uint32; comparisons need EventType.X.value not the
  enum object directly
- Rays on the image plane appear as HIT_ELEMENT, not ABSORBED
- Remove fix_seed/DeviceType references (not in current PyPI release)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GXnPW72q5DnUoApAU7mvLL
Comment thread tools/build_docs.sh

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer a script written in python instead of bash. The dev dependencies should be available if it is executed like this uv run tools/build_docs.py

@valentinstoecker

valentinstoecker commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

when you're done with the requested changes please run uv sync to add the new dependencies to the lockfile

@Atraxus Atraxus removed their request for review July 10, 2026 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants